I have created 2 interfaces and substituted in place of GitItem, fixes the alignment issue but unable to get the icon property. Need some help to figure it out.

Thank you for your PR!

We use if (item is ShortcutItem) {} a lot so it's important to inherit from ShortcutItem class:

public sealed class GitShortcutItem : ShortcutItem, GitItem {}

Okay, should I discard the previous changes?

Just like this:

if (isSymLink)
{
    if (isGitRepo) { return new() { ... }; }
    else { return new() { ... }; }
}

Thank you for your PR!

We use if (item is ShortcutItem) {} a lot so it's important to inherit from ShortcutItem class:

public sealed class GitShortcutItem : ShortcutItem, GitItem {}

To inherit from ShortcutItem and GitItem I'll have to remove the sealed keyword. This is the reason I created interfaces.

Okay, should I discard the previous changes?

Just like this:

if (isSymLink)

{

    if (isGitRepo) { return new() { ... }; }

    else { return new() { ... }; }

}

Seal was added for IL compiler performance aspects, you can remove any time you want.

Seal was added for IL compiler performance aspects, you can remove any time you want.

Okay. Understood, thank you. Will do today.

You don't need interfaces you introduced.

You don't need interfaces you introduced.

I won't be able to inherit ShortcutItem and GitItem together. That's why I used interfaces.

Ahh you're absolutely right, I forgot that.

Can you change "is/as ShortcutItem" to IShortcutItem in everywhere as well in that case? Then you dint have to inherit ShortcutItem and GitItem but should inherit these two interfaces.

Ahh you're absolutely right, I forgot that.

Can you change "is/as ShortcutItem" to IShortcutItem in everywhere as well in that case? Then you dint have to inherit ShortcutItem and GitItem but should inherit these two interfaces.

Sure thing I'll carefully substitute IShortcutItem in the place of ShortcutItem.

I have replaced all "Is Shortcut" Item with interface so that it checks for the interface properties, I have not changes "as ShortcutItem" because it might want to actually create an Instance of the class rather than the Interface as the ShortcutItem is being derived from ListedItem class also.

Ahh you're absolutely right, I forgot that.

Can you change "is/as ShortcutItem" to IShortcutItem in everywhere as well in that case? Then you dint have to inherit ShortcutItem and GitItem but should inherit these two interfaces.

I've made the commits! :)